Skip to content

fix: [iOS] 내 서재 > 도서 없는 케이스 서브 텍스트 상이 #256

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
doyeonk429 merged 1 commit into from
Nov 9, 2025
Merged

fix: [iOS] 내 서재 > 도서 없는 케이스 서브 텍스트 상이 #256

doyeonk429 merged 1 commit into from
Nov 9, 2025

Conversation

@doyeonk429
Copy link
Member

@doyeonk429 doyeonk429 commented Nov 8, 2025 •
edited by coderabbitai bot
Loading

🔗 관련 이슈

📘 작업 유형

  • ✨ Feature (기능 추가)
  • 🐞 Bugfix (버그 수정)
  • 🔧 Refactor (코드 리팩토링)
  • ⚙️ Chore (환경 설정)
  • 📝 Docs (문서 작성 및 수정)
  • ✅ Test (기능 테스트)
  • 🎨 style (코드 스타일 수정)

📙 작업 내역

  • 서브타이틀 문구 수정

🧪 테스트 내역

  • 브라우저/기기에서 동작 확인
  • 엣지 케이스 테스트 완료
  • 기존 기능 영향 없음

🎨 스크린샷 또는 시연 영상 (선택)

Simulator Screenshot - ReedDefaultSize - 2025-11-08 at 09 50 30 Simulator Screenshot - ReedDefaultSize - 2025-11-08 at 09 50 41

✅ PR 체크리스트

  • 커밋 메시지가 명확합니다
  • PR 제목이 컨벤션에 맞습니다
  • 관련 이슈 번호를 작성했습니다
  • 기능이 정상적으로 작동합니다
  • 불필요한 코드를 제거했습니다

💬 추가 설명 or 리뷰 포인트 (선택)

  • 큰 이슈 없습니다.

Summary by CodeRabbit

릴리스 노트

  • 개선 사항
    • 아카이브 화면의 빈 상태 메시지를 개선하여 사용자에게 더 명확한 가이드를 제공합니다.

@doyeonk429 doyeonk429 self-assigned this Nov 8, 2025
@doyeonk429 doyeonk429 added the 🐞 fix Something isn't working label Nov 8, 2025
@github-actions github-actions bot requested a review from clxxrlove November 8, 2025 00:55
@coderabbitai
Copy link

coderabbitai bot commented Nov 8, 2025 •
edited
Loading

Walkthrough

내 서재 비어있을 때 표시되는 빈 상태 뷰의 부제목 텍스트를 수정합니다. 기존 "검색해서 책을 등록해보세요"에서 "도서 등록 후 나만의 아카이브를 만들어보세요"로 변경되며, 로직이나 공개 API에는 변경이 없습니다.

Changes

Cohort / File(s) 변경 사항
내 서재 빈 상태 뷰 텍스트 수정
src/Projects/BKPresentation/Sources/ArchiveFlow/View/EmptyStateView.swift		 EmptyStateView 상수의 부제목 텍스트를 "검색해서 책을 등록해보세요"에서 "도서 등록 후 나만의 아카이브를 만들어보세요"로 변경

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2분

  • 문자열 상수 변경으로 코드 로직에 영향 없음
  • 단일 파일의 간단한 텍스트 수정

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 풀 리퀘스트 제목이 변경 사항의 주요 내용을 명확하게 요약하고 있습니다.
Linked Issues check ✅ Passed 코드 변경 사항이 연결된 이슈 #255의 요구사항을 충족하며, 빈 서재 경우의 서브타이틀 텍스트를 정확히 수정했습니다.
Out of Scope Changes check ✅ Passed 모든 변경 사항이 빈 서재 서브타이틀 텍스트 수정이라는 범위 내에 있으며, 범위를 벗어난 변경은 없습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch BOOK-423-fix/#255
📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e46cf91 and deb8b71.

📒 Files selected for processing (1)
  • src/Projects/BKPresentation/Sources/ArchiveFlow/View/EmptyStateView.swift (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings 
Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 163
File: src/Projects/BKData/Sources/DTO/Request/UserBookRegisterRequestDTO.swift:7-9
Timestamp: 2025-08-08T16:13:31.712Z
Learning: Reed-iOS에서 서버 스펙 변경으로 DTO 필드명이 바뀔 때, doyeonk429는 코드 전반의 파라미터/시그니처 리네이밍까지 확장하지 않고, 실제 요청 직렬화 키가 맞는지(예: isbn13)만 보장하면 충분하다는 범위 최소화 방식을 선호한다. (PR: YAPP-Github/Reed-iOS#163, 파일: BKData/Sources/DTO/Request/UserBookRegisterRequestDTO.swift)

Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 157
File: src/Projects/BKPresentation/Sources/MainFlow/Home/View/HomeViewController.swift:19-24
Timestamp: 2025-08-08T01:38:59.656Z
Learning: doyeonk429는 Reed-iOS 프로젝트에서 접근성(accessibility) 관련 개선사항은 현재 작업 중인 PR에서 즉시 처리하지 않고, 접근성 전용 PR이나 이슈를 별도로 만들어 한번에 처리하는 것을 선호한다.

Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 157
File: src/Projects/BKPresentation/Sources/AuthFlow/View/LoginView.swift:43-45
Timestamp: 2025-08-08T01:39:15.620Z
Learning: doyeonk429는 Reed-iOS 프로젝트에서 접근성 개선 작업을 별도의 전용 PR이나 이슈에서 일괄 처리하는 것을 선호한다. 개별 기능 구현 PR에서는 접근성 관련 제안을 하지 않고, 접근성 전담 작업에서 한번에 처리하는 방식을 원한다.

Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 68
File: src/Projects/BKDesign/PreviewApp/Sources/View/BKButtonTestViewController.swift:124-133
Timestamp: 2025-07-10T08:21:49.399Z
Learning: doyeonk429는 테스트 전용으로만 사용되는 extension이나 코드는 해당 테스트 파일에 그대로 두는 것을 선호합니다. 실제 프로덕션 코드에서 사용되지 않는 테스트 전용 코드는 별도 파일로 분리하지 않고 테스트 파일 내에 유지하는 것이 그들의 코드 구성 방식입니다.

Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 138
File: src/Projects/BKDesign/Resources/Assets.xcassets/icons/home.imageset/Contents.json:4-14
Timestamp: 2025-08-04T15:20:43.982Z
Learning: doyeonk429는 Reed-iOS 프로젝트에서 에셋 파일명이 영어가 아닌 경우(한글, 일본어 등)에는 경고를 받고 싶어하지만, 영어 파일명의 네이밍 컨벤션(예: home.png vs home1.png)은 기능적으로 문제없다면 신경쓰지 않는 것을 선호한다.

Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 98
File: src/Projects/BKPresentation/Sources/AuthFlow/View/TermsView.swift:78-85
Timestamp: 2025-07-22T05:37:28.756Z
Learning: doyeonk429는 Reed-iOS 프로젝트에서 더미 데이터나 테스트 데이터의 구체적인 내용(예: URL 주소)에 대해서는 리뷰 시 지적하지 않기를 선호한다. 더미 데이터는 임시적이며 실제 기능 구현 시 교체될 예정이므로 URL의 구체적인 주소는 중요하지 않다고 본다.
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: build
🔇 Additional comments (1)
src/Projects/BKPresentation/Sources/ArchiveFlow/View/EmptyStateView.swift (1)

72-72: LGTM! 서브타이틀 문구가 명확하게 수정되었습니다.

빈 서재 화면에서 회원에게 표시되는 서브타이틀이 "도서 등록 후 나만의 아카이브를 만들어보세요"로 변경되어 사용자에게 더 명확한 가이드를 제공합니다. 텍스트만 변경되었고 로직에는 영향이 없습니다.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@doyeonk429 doyeonk429 merged commit dcc0821 into develop Nov 9, 2025
5 checks passed
@doyeonk429 doyeonk429 deleted the BOOK-423-fix/#255 branch November 9, 2025 00:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@clxxrlove clxxrlove clxxrlove approved these changes

Assignees

@doyeonk429 doyeonk429

Labels

🐞 fix Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[BOOK-423/qa] [iOS] 내 서재 > 도서 없는 케이스 서브 텍스트 상이

3 participants

@doyeonk429 @clxxrlove